docs(poznote): sync 6.30.2 default#384
Merged
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughUpdated Poznote chart documentation to reference image tag ChangesPoznote Documentation Update
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/pages/docs/charts/poznote.mdx (1)
149-162: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueConsider cross-linking Backup and Upgrade Notes sections.
The new Upgrade Notes section duplicates the PVC backup guidance already stated in the Backup section above (Line 151). Consider referencing the Backup section instead of restating the guidance to avoid drift if the backup instructions change later.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/pages/docs/charts/poznote.mdx` around lines 149 - 162, The Backup guidance is repeated in the Upgrade Notes section, so update the docs in the poznote.mdx content to avoid duplicating the same PVC backup instructions. Keep the detailed backup steps under the Backup heading and change the Upgrade Notes text to refer back to that section using the existing section titles rather than restating the PVC backup details.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/pages/docs/charts/poznote.mdx`:
- Line 13: The Poznote docs are referencing an image tag and upgrade notes that
do not match a public release, so update the `poznote.mdx` content to use the
published Poznote version and matching notes. In the docs section that mentions
the `ghcr.io/timothepoznanski/poznote` image, replace the `6.30.2` tag with the
actual release version `6.30.0`, and make sure the accompanying Upgrade Notes
text is aligned with that release. Use the existing Poznote chart description in
`poznote.mdx` to locate and correct the versioned release details.
---
Nitpick comments:
In `@src/pages/docs/charts/poznote.mdx`:
- Around line 149-162: The Backup guidance is repeated in the Upgrade Notes
section, so update the docs in the poznote.mdx content to avoid duplicating the
same PVC backup instructions. Keep the detailed backup steps under the Backup
heading and change the Upgrade Notes text to refer back to that section using
the existing section titles rather than restating the PVC backup details.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: e5f425d4-635f-4165-a6d0-f25af40f8640
📒 Files selected for processing (1)
src/pages/docs/charts/poznote.mdx
Contributor
Author
|
Handled the CodeRabbit top-level nitpick in 54a20d4. Adjusted the Poznote Upgrade Notes to reference the Backup section instead of duplicating PVC backup guidance, keeping the detailed backup instructions in one place. Validation:
This CodeRabbit item was posted only in the review summary, so there is no resolvable review thread to close. |
mberlofa
added a commit
to helmforgedev/charts
that referenced
this pull request
Jul 8, 2026
## Summary - Bump Poznote from 6.29.0 to 6.30.2. - Update chart defaults, tests, README, and DESIGN notes. - Add upgrade notes for the UI, backup, API, security hardening, MCP, dark mode, and mobile task fixes. Closes #703. ## Upstream Evidence - Issue target was 6.29.2, but upstream already published 6.30.2 as the latest stable release on 2026-07-07, so this PR updates to the current stable tag instead of opening a stale bump. - Release: https://github.com/timothepoznanski/poznote/releases/tag/6.30.2 - Release state: stable, not draft, not prerelease; published 2026-07-07. - Image verified: ghcr.io/timothepoznanski/poznote:6.30.2 - Manifest platforms verified: linux/amd64, linux/arm64. - Relevant upstream notes across 6.29.2 through 6.30.2: UI improvements, MCP server fix, mobile task edit fixes, brute-force login throttling, backup import validation, trashed notes included in exports, new REST API endpoints, and dark mode styling fixes. No Kubernetes-facing configuration change was identified. ## Site Sync - helmforgedev/site#384 ## Validation - make image-verify IMAGE=ghcr.io/timothepoznanski/poznote:6.30.2 - make deps-check CHART=poznote - helm unittest charts/poznote: 10 suites, 63 tests - make standards-check CHART=poznote - node scripts/charts/template-standards-check.js --chart poznote - make site-sync-check CHART=poznote - make validate-chart CHART=poznote: passed 16 layers, including k3d behavioral validation for default and all Poznote CI scenarios - make release-check REPO=charts - make attribution-check REPO=charts <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Updated the default Poznote version to **6.30.2** across the chart. * Added upgrade guidance, including a reminder to back up the `data` volume before upgrading. * **Documentation** * Refreshed chart docs and design notes to match the new release version and image tag. * **Tests** * Updated chart rendering checks to validate the new default container image. <!-- end of auto-generated comment: release notes by coderabbit.ai --> Co-authored-by: MergeCheck <mergecheck@example.invalid>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Validation
Charts PR: helmforgedev/charts#722
Summary by CodeRabbit
6.30.2.